Skip to content

Conversation

@waynemo
Copy link
Contributor

@waynemo waynemo commented Oct 1, 2020

No description provided.

@waynemo waynemo marked this pull request as ready for review October 1, 2020 18:51
@waynemo waynemo requested a review from ankitarorabit October 1, 2020 18:51
@waynemo waynemo merged commit 4bac9a3 into Azure:feature/communication/phonenumber Oct 1, 2020
ankitarorabit pushed a commit to ankitarorabit/azure-sdk-for-java that referenced this pull request Oct 1, 2020
…ger (Azure#15861)

* Update swagger and regenerate code

* Fix tests after code regeneration
RezaJooyandeh pushed a commit that referenced this pull request Oct 2, 2020
* Azure Communication Service - Phone Number Administration SDK (#15497)

* Add Phone Number functionality to azure-communication-administration

* Update checkstyle suppressions

* Azure Communication Services - Refactor PhoneNumberAsyncClient to Wrap Exceptions (#15794)

* Catch exceptions and return error Mono/PagedFlux

* Refactor PhoneNumberAsyncClient method overloads to consolidate logic

* Azure Communication Service - Update Phone Number Admin Null Checks (#15811)

* Add null checks for required method arguments

* Remove setter null check to allow clearing of httpPipeline

* Updating azure-communication-administration README with a few samples (#15816)

* Updating javadoc with a few samples

* resolve comments

* Fixing spot check errors

* fix typo

* Add Env Var check to skip PhoneNumber integration tests (#15855)

* Samples and Readme update for Administration package (#15853)

* More samples and readme changes

* PR comments

* build failure

* fix indentation

* azure-communication-administration - Regenerate Code for Updated Swagger (#15861)

* Update swagger and regenerate code

* Fix tests after code regeneration

* Fix the param name

* updating the sample code

Co-authored-by: waynemo <wamo@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants